Skip to content

Dummy PR to test coverage test changes in CI #114917

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Dummy PR to test coverage test changes in CI #114917

wants to merge 2 commits into from

Conversation

@Zalathar
Copy link
Contributor

The x86_64-gnu-llvm-15 job that normally runs on PRs doesn't include the profiler runtime, so most of the coverage tests don't get run on PRs.

(That's why #114875 didn't fail until an actual merge was attempted.)

r? @ghost

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Aug 17, 2023
@Zalathar
Copy link
Contributor Author

@rustbot experimental

@Zalathar
Copy link
Contributor Author

@rustbot author

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 17, 2023
@Zalathar
Copy link
Contributor Author

Zalathar commented Aug 17, 2023 •
edited
Loading

Success!

https://github.com/rust-lang/rust/actions/runs/5885191281

@Zalathar
Copy link
Contributor Author

Now testing #114843 in CI on Linux/Windows/macOS, to ensure I don't get any nasty surprises after it's reviewed.

@Zalathar
Copy link
Contributor Author

Success:

@bors
Copy link
Collaborator

bors commented Aug 29, 2023

☔ The latest upstream changes (presumably #115183) made this pull request unmergeable. Please resolve the merge conflicts.

@Zalathar Zalathar force-pushed the dummy-coverage branch 2 times, most recently from b605cc7 to c4511d7 Compare September 2, 2023 00:08
@Zalathar
Copy link
Contributor Author

Zalathar commented Sep 2, 2023

Testing #114843 on Linux/Windows/macOS (coverage-map and run-coverage only), to see if everything still works on the big 3 configurations.

@rust-log-analyzer

This comment has been minimized.

Sign in to view
@Zalathar
Copy link
Contributor Author

Zalathar commented Sep 2, 2023

Those Linux failures look like the same ones I saw on the aarch64-gnu build, so hopefully that's a platform issue and I don't have to worry about ARM Linux specifically.

I'm going to try another run of Windows only, to flush out any unknown issues on that platform.

@rust-log-analyzer

This comment has been minimized.

Sign in to view
@Zalathar
Copy link
Contributor Author

Zalathar commented Sep 2, 2023

Test failures on Windows seem to be the same as the Linux ones, which is surprising.

Perhaps there is some common property of the main Linux/Windows CI environments that is not shared by x86_64-gnu-llvm-15 or my local Aarch64 macOS environment.

@Zalathar
Copy link
Contributor Author

Zalathar commented Sep 2, 2023

I'll try a CI macOS build to double check all the major platforms.

@rust-log-analyzer

This comment has been minimized.

Sign in to view
@Zalathar
Copy link
Contributor Author

Zalathar commented Sep 2, 2023

Hmm, so the overall results so far seem to be:

Failure (probably the same failures, though I haven't fully verified this):

  • aarch64-gnu
  • x86_64-gnu
  • x86_64-msvc
  • x86_64-apple-1

Success:

  • x86_64-gnu-llvm-15
  • Local build on aarch64 macOS
  • Local build on aarch64 macOS (--stage=2)

@Zalathar
Copy link
Contributor Author

Zalathar commented Sep 2, 2023

Trying another build with the default PR CI config (x86_64-gnu-llvm-15) just to confirm again that that one still works.

@rust-log-analyzer

This comment has been minimized.

Sign in to view
@rust-log-analyzer

This comment has been minimized.

Sign in to view
@rust-log-analyzer

This comment has been minimized.

Sign in to view
@Zalathar Zalathar mentioned this pull request Oct 26, 2024
Merged
@Zalathar Zalathar force-pushed the dummy-coverage branch 2 times, most recently from 8e96a9c to 13a1093 Compare October 27, 2024 00:40
@rust-log-analyzer

This comment has been minimized.

Sign in to view
@rust-log-analyzer

This comment has been minimized.

Sign in to view
@Zalathar Zalathar force-pushed the dummy-coverage branch 3 times, most recently from 1d9b95b to 2010de3 Compare November 3, 2024 22:35
@bors
Copy link
Collaborator

bors commented Nov 9, 2024

☔ The latest upstream changes (presumably #132584) made this pull request unmergeable. Please resolve the merge conflicts.

@bors bors added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Nov 9, 2024
@rustbot rustbot added the A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. label Sep 18, 2025
@Zalathar
Copy link
Contributor Author

@bors try jobs=x86_64-msvc-1,x86_64-msvc-2

@rust-bors

This comment has been minimized.

Sign in to view
rust-bors bot added a commit that referenced this pull request Sep 18, 2025
@rust-bors
Auto merge of #114917 - Zalathar:dummy-coverage, r=<try>
75f1db9 
Dummy PR to test coverage test changes in CI

try-job: x86_64-msvc-1
try-job: x86_64-msvc-2
@rust-log-analyzer

This comment has been minimized.

Sign in to view
@Zalathar
Copy link
Contributor Author

@bors try cancel

@rust-bors
Copy link

rust-bors bot commented Sep 18, 2025

Try build cancelled. Cancelled workflows:

@Zalathar
Copy link
Contributor Author

@bors try jobs=x86_64-msvc-1,x86_64-msvc-2

@rust-bors

This comment has been minimized.

Sign in to view
rust-bors bot added a commit that referenced this pull request Sep 18, 2025
@rust-bors
Auto merge of #114917 - Zalathar:dummy-coverage, r=<try>
c6c118b 
Dummy PR to test coverage test changes in CI

try-job: x86_64-msvc-1
try-job: x86_64-msvc-2
@rust-bors
Copy link

rust-bors bot commented Sep 18, 2025

☀️ Try build successful (CI)
Build commit: c6c118b (c6c118bb1e1008a1303c75dbecb39e1789670759, parent: 93117677d857bb7c3f12c9dc500d77839f8fb13d)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. A-run-make Area: port run-make Makefiles to rmake.rs A-testsuite Area: The testsuite used to check the correctness of rustc S-experimental Status: Ongoing experiment that does not require reviewing and won't be merged in its current state. S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Zalathar @bors @rust-log-analyzer @rustbot @Dylan-DPC